Skip to content

test: cover flag utilities - #1050

Closed
w3lld1 wants to merge 1 commit into
typescript-eslint:mainfrom
w3lld1:test/flags-utilities
Closed

test: cover flag utilities#1050
w3lld1 wants to merge 1 commit into
typescript-eslint:mainfrom
w3lld1:test/flags-utilities

Conversation

@w3lld1

@w3lld1 w3lld1 commented Jul 10, 2026

Copy link
Copy Markdown

PR Checklist

Overview

Adds focused true/false coverage for every exported flag utility in src/flags.ts, including modifier, node, object, symbol, transient-symbol-link, and type flags.

Validation:

  • pnpm exec vitest run src/flags.test.ts (6 passed)
  • pnpm test --run (321 passed)
  • pnpm lint
  • pnpm tsc --noEmit
  • pnpm build
  • git diff --check

💖

@codecov

codecov Bot commented Jul 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.82%. Comparing base (e59a11b) to head (bf3aa9b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1050   +/-   ##
=======================================
  Coverage   58.82%   58.82%           
=======================================
  Files          29       29           
  Lines         901      901           
  Branches      283      283           
=======================================
  Hits          530      530           
+ Misses        287      286    -1     
- Partials       84       85    +1     
Flag Coverage Δ
5.5.4 58.82% <ø> (+0.11%) ⬆️
5.9.3 58.82% <ø> (+0.11%) ⬆️
latest ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JoshuaKGoldberg

Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🧪 Tests: Add tests for remaining functions in src/flags.ts

2 participants